Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build and push nuget artifacts #3110

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Build and push nuget artifacts #3110

merged 2 commits into from
Nov 20, 2023

Conversation

pavlovic-ivan
Copy link
Collaborator

@pavlovic-ivan pavlovic-ivan commented Nov 20, 2023

What type of PR is this?

This is an improvement PR

What this PR does / why we need it:

This PR add workflow jobs that pack and push nuget artifacts, Armada clients

Special notes for your reviewer:

Magefile target that was doing both pack and push (named PushNuget), has been refactored to only pack nuget, thus implementing single responsibility principle.

Prerequisites:

  • create a protected environment called nuget-release with the NUGET_API_KEY secret

Copy link
Member

@dave-gantenbein dave-gantenbein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jgiannuzzi
Copy link
Member

The nuget-release environment with the NUGET_API_KEY secret needs to be created before this PR gets merged. I'm on it!

@pavlovic-ivan
Copy link
Collaborator Author

The nuget-release environment with the NUGET_API_KEY secret needs to be created before this PR gets merged. I'm on it!

I will add this to the PR desc

@jgiannuzzi
Copy link
Member

Environment and secret created + PR description updated

* Add push nuget workflow job. Refactor magefile to only pack nuget not pack&push

Signed-off-by: Ivan Pavlovic <[email protected]>
@severinson severinson enabled auto-merge (squash) November 20, 2023 15:01
@severinson severinson merged commit c7681fb into armadaproject:master Nov 20, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants